Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated query of git version string, Updated vs-build contents #45

Merged
16 commits merged into from
Sep 12, 2017
Merged

Updated query of git version string, Updated vs-build contents #45

16 commits merged into from
Sep 12, 2017

Conversation

HaymanConsulting
Copy link
Contributor

This includes the relocation of RANLUX.f90 from the icefloe module to the nwtc_library so that other modules can point to this code.

Greg Hayman added 12 commits August 14, 2017 10:00
Merge remote-tracking branch 'origin/dev' into dev
d37e036
Added back Stack Reserve setting for all x64 builds
bc2faeb
Draft of Binary output without compression
f619aa8 
Added FileID = FileFmtID_NoCompressWithoutTime which would now branch the file reading code to correctly parse the new uncompressed file format, so all readers need to be updated to support this new format, but old readers still work with legacy binary files.
Working version of the uncompressed binary file format
8627bd5 
The user can get uncompressed binary by setting the OutFileFmt option in the primary FAST input file to 0.  This generates a text version of the output and an uncompressed binary version.
Updated SubDyn project to use stack reserve size for x64 builds
ff87ce3
Merge remote-tracking branch 'origin/dev' into dev
6cb5a2a
Added QueryGitHash() to NWTC_IO for reading the git commit hash string
beba307 
Refactored code to obtain the git commit hash so the a single function call returns the commit string regardless of the build approach
Add git commit string support, new version support for various drivers
abcb7b8
Fixed some compiling issues for double precision
ac7079f
Modified the code related to obtaining the GIT version
4c567b7 
Based on input for Bonnie Jonkman, I have altered the implementation for obtaining the git-related version string.
Moved ranlux.f90 into the nwtc_library and out of icefloe
5137119 
Now ranlux.f90 can be used by other modules
Merge remote-tracking branch 'origin/dev' into dev
2c9a53b
@ghost
Copy link

ghost commented Sep 11, 2017

  • is it manageable to change the executable name from the vs build to openfast.exe?
  • the visual studio generated build files should be gitignored so that simply building the executable doesn't cause the version to be "dirty"
(C:\Users\rmudafor\AppData\Local\Continuum\Anaconda3) C:\Users\rmudafor\Desktop\openfast>git st
HEAD detached at greg/dev
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   vs-build/MAPlib/MAP_dll.vcxproj
        modified:   vs-build/Registry/FAST_Registry.vcxproj

Untracked files:
  (use "git add <file>..." to include in what will be committed)

        vs-build/FAST/FAST.VC.VC.opendb
        vs-build/FAST/FAST.VC.db
        vs-build/Registry/Debug_Double/

no changes added to commit (use "git add" and/or "git commit -a")

Greg Hayman added 4 commits September 11, 2017 14:25
Going back to ignoring the entire vs-build folder in .gitignore
b294ed1
Bug Fix: Added back DOUBLE_PRECISION preprocessor flag for some builds
504c430 
Also change output executables to start with openfast_
Fixed vs-build issues: DOUBLE_PRECISION, /warn:interfaces
62b7bef 
Correctly added back the /DDOUBLE_PRECISION flag for stand-along driver projects, and removed the /warn:interfaces option
vs-build fixes for UnsteadyAero
4302859 
added back /DDOUBLE_PRECISION flag and corrected default float size for dierckx_fitpack.f when building double precision
@ghost ghost merged commit 4c8ea3a into OpenFAST:dev Sep 12, 2017
@Christophe-delFosse Christophe-delFosse mentioned this pull request Apr 8, 2021
Closed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaelasprague michaelasprague Awaiting requested review from michaelasprague

Assignees

@michaelasprague michaelasprague

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@HaymanConsulting @michaelasprague